Best reviewed: commit by commit

Powered by Pull Assistant. Last update 34e00ba ... fed7e78. Read the comment docs.

Codecov Report

❌ Patch coverage is 31.74603% with 43 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.40%. Comparing base (6b95611) to head (fed7e78).
⚠️ Report is 293 commits behind head on master.

@@            Coverage Diff             @@
##           master     #104      +/-   ##
==========================================
- Coverage   50.34%   49.40%   -0.95%     
==========================================
  Files          21       22       +1     
  Lines        1289     1352      +63     
  Branches      169      174       +5     
==========================================
+ Hits          649      668      +19     
- Misses        628      672      +44     
  Partials       12       12              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Hello @arokem, Thank you for updating!

• In the file dmriprep/utils/register.py:

Line 79:5: F841 local variable 'transformed' is assigned to but never used
Line 134:1: F811 redefinition of unused 'AffineRegistration' from line 10
Line 137:9: F841 local variable 'nbins' is assigned to but never used
Line 138:9: F841 local variable 'sampling_prop' is assigned to but never used
Line 139:9: F841 local variable 'metric' is assigned to but never used
Line 140:9: F841 local variable 'level_iters' is assigned to but never used
Line 141:9: F841 local variable 'sigmas' is assigned to but never used
Line 142:9: F841 local variable 'factors' is assigned to but never used
Line 143:9: F841 local variable 'pipeline' is assigned to but never used

To test for issues locally, pip install flake8 and then run flake8 dmriprep.

Comment last updated at 2020-09-29 15:30:40 UTC

My plan is to rebase this on top of #78 after Derek updates that branch.

see #78 (comment)